Skip to content

Snowshare: fix update script - #11726

Merged
MickLesk merged 3 commits into
community-scripts:mainfrom
TuroYT:main
Feb 11, 2026
Merged

Snowshare: fix update script#11726
MickLesk merged 3 commits into
community-scripts:mainfrom
TuroYT:main

Conversation

@TuroYT

@TuroYT TuroYT commented Feb 9, 2026

Copy link
Copy Markdown
Contributor

✍️ Description

fix the update script to clean old generated content before update

🔗 Related Issue

Fixes #

✅ Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.

🛠️ Type of Change (X in brackets)

  • 🐞 Bug fix – Resolves an issue without breaking functionality.
  • ✨ New feature – Adds new, non-breaking functionality.
  • 💥 Breaking change – Alters existing functionality in a way that may require updates.
  • 🆕 New script – A fully functional and tested script or script set.
  • 🌍 Website update – Changes to website-related JSON files or metadata.
  • 🔧 Refactoring / Code Cleanup – Improves readability or maintainability without changing functionality.
  • 📝 Documentation update – Changes to README, AppName.md, CONTRIBUTING.md, or other docs.

@TuroYT
TuroYT requested a review from a team as a code owner February 9, 2026 11:14
@github-actions github-actions Bot added bugfix update script A change that updates a script labels Feb 9, 2026
@MickLesk

MickLesk commented Feb 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

We can Just add CLEAN_INSTALL=1 before the fetch_and_deploy. This remove the whole Installation, but i need your Input, which Files should be backed up before?

@MickLesk

MickLesk commented Feb 10, 2026

Copy link
Copy Markdown
Member

@TuroYT feedback?

@TuroYT

TuroYT commented Feb 10, 2026

Copy link
Copy Markdown
Contributor Author

Sorry for the late reply. A folder called 'uploads' needs to be kept, and the rest can be deleted.

@TuroYT
TuroYT marked this pull request as draft February 10, 2026 14:18
@TuroYT

TuroYT commented Feb 11, 2026

Copy link
Copy Markdown
Contributor Author

@MickLesk how i can implements the CLEAN_INSTALL=1 and keep the uploads dir ?

@TuroYT TuroYT mentioned this pull request Feb 11, 2026
Closed
@MickLesk
Implement upload backup and restore in update script
198dc2f 
Added backup and restore functionality for uploads during the Snowshare update process.
@MickLesk

MickLesk commented Feb 11, 2026

Copy link
Copy Markdown
Member

done

@MickLesk
MickLesk marked this pull request as ready for review February 11, 2026 09:32
@MickLesk
MickLesk merged commit f34f994 into community-scripts:main Feb 11, 2026
1 check passed
@TuroYT

TuroYT commented Feb 11, 2026

Copy link
Copy Markdown
Contributor Author

thanks !

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Feb 15, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@MickLesk MickLesk MickLesk approved these changes
@tremor021 tremor021 tremor021 approved these changes

Assignees

No one assigned

Labels

bugfix update script A change that updates a script

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@TuroYT @MickLesk @tremor021